New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

update dc.op() with dc.op_with_named_attrs() #942

Draft

prempk229 wants to merge 1 commit into tenstorrent:main from prempk229:prempk/update_dcop_with_named_attrs

prempk229 commented Dec 19, 2024

Fixes : #220

Updated function name of dc.op() with dc.op_with_named_attrs() and added a parameter corresponding to the op.
Validated the function call dc.op_with_named_attrs() for "reshape", "softmax" in test_forge_ops.py

Changes are completed for decomposing_context. For lowering_context, necessary changes are in place already.


          update dc.op() with dc.op_with_named_attrs()

58af4ad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

nvukobratTT Awaiting requested review from nvukobratTT nvukobratTT will be requested when the pull request is marked ready for review nvukobratTT is a code owner

pilkicTT Awaiting requested review from pilkicTT pilkicTT will be requested when the pull request is marked ready for review pilkicTT is a code owner

dgolubovicTT Awaiting requested review from dgolubovicTT dgolubovicTT will be requested when the pull request is marked ready for review dgolubovicTT is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet